Skip to content

Conversation

marcoieni
Copy link
Member

@marcoieni marcoieni commented May 15, 2025

This job is slow. We want to split it to make it faster.
Plus, we remove one large runner from CI.
r? @Kobzol

try-job: dist-ohos-*

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels May 15, 2025
@marcoieni
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request May 15, 2025
ci: split the dist-ohos job

try-job: `dist-ohos-*`
@bors
Copy link
Collaborator

bors commented May 15, 2025

⌛ Trying commit 839534e with merge 591cdad...

@bors
Copy link
Collaborator

bors commented May 15, 2025

☀️ Try build successful - checks-actions
Build commit: 591cdad (591cdade90347ca281d574f419a5406c33400d57)

@marcoieni marcoieni marked this pull request as ready for review May 15, 2025 15:30
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 15, 2025
@marcoieni
Copy link
Member Author

@rustbot ready

@Kobzol
Copy link
Member

Kobzol commented May 15, 2025

Seems pretty straightforward, thanks. I wonder why the ohos dist builders build src tarballs, but apparently multiple dist builders do that, even though it should be only done on x86-64. Anyway, that's a separate thing.

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented May 15, 2025

📌 Commit 839534e has been approved by Kobzol

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 15, 2025
hkBst added a commit to hkBst/rust that referenced this pull request May 16, 2025
ci: split the dist-ohos job

try-job: `dist-ohos-*`
bors added a commit to rust-lang-ci/rust that referenced this pull request May 17, 2025
…iaskrgr

Rollup of 8 pull requests

Successful merges:

 - rust-lang#140208 (Make well-formedness predicates no longer coinductive)
 - rust-lang#140957 (Add `#[must_use]` to Array::map)
 - rust-lang#141031 (Async drop fix for dropee from another crate (rust-lang#140858))
 - rust-lang#141036 (ci: split the dist-ohos job)
 - rust-lang#141051 (Remove some unnecessary erases)
 - rust-lang#141056 (Lowercase git url for rust-lang/enzyme.git)
 - rust-lang#141059 (HIR: explain in comment why `ExprKind::If` "then" is an `Expr`)
 - rust-lang#141070 (Do not emit help when shorthand from macro when suggest `?` or `expect`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 0031282 into rust-lang:master May 17, 2025
7 checks passed
@rustbot rustbot added this to the 1.89.0 milestone May 17, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request May 17, 2025
Rollup merge of rust-lang#141036 - marcoieni:split-dist-ohos, r=Kobzol

ci: split the dist-ohos job

try-job: `dist-ohos-*`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants